-
Notifications
You must be signed in to change notification settings - Fork 667
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[gui-tests][full-ci] retry failed scenario #11920
base: master
Are you sure you want to change the base?
Conversation
d6df6e6
to
cc4ed54
Compare
a93e55d
to
7161ffa
Compare
7161ffa
to
9a2b9bc
Compare
9a2b9bc
to
3b4d16e
Compare
@@ -73,12 +73,14 @@ def get_default_home_dir(): | |||
'clientConfigDir': 'CLIENT_CONFIG_DIR', | |||
'guiTestReportDir': 'GUI_TEST_REPORT_DIR', | |||
'ocis': 'OCIS', | |||
'screenRecordOnFailure': 'SCREEN_RECORD_ON_FAILURE', | |||
'recordVideoOnFailure': 'RECORD_VIDEO_ON_FAILURE', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how about writing these attributes in snake_case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would be great. but not in this PR.
} | ||
|
||
DEFAULT_PATH_CONFIG = { | ||
'custom_lib': os.path.abspath('../shared/scripts/custom_lib'), | ||
'home_dir': get_default_home_dir(), | ||
# allow to record first 5 videos | ||
'videoRecordLimit': 5, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'videoRecordLimit': 5, | |
'video_record_limit': 5, |
Implements retry failing test scenario.
Condition to trigger retry:
full-ci
in the PR titileNote
--retry
and--abortOnFail
options clash with each other.So, we enable
--abortOnFail
on normal PRand enable
--retry
if above conditions are metWhat happens when retry is triggered?
Other changes:
SCREEN_RECORD_ON_FAILURE
toRECORD_VIDEO_ON_FAILURE
screenRecordOnFailure
torecordVideoOnFailure
videoRecordLimit
to limit video recordings of failed scenarios -default: 5
(because of large size)ReportHelper
file